-
Notifications
You must be signed in to change notification settings - Fork 6.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(ripples): support updating global ripple options at runtime #14705
feat(ripples): support updating global ripple options at runtime #14705
Conversation
This comment has been minimized.
This comment has been minimized.
bdc9238
to
e32f201
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The approach looks good, but is it really something we'd want to support? I don't know how common it would be for an app to want to disable/enable ripples after it has been initialized.
@crisbeto Since this is already possible right now, but just breaks for our custom Also since it has been requested and I personally think it's valuable and didn't even require some changes to |
* Allows updating any global ripple option at runtime. This makes it possible for developers to disable ripples at runtime. Closes angular#9729
e32f201
to
665c3ea
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…ular#14705) * feat(ripples): support updating global ripple options at runtime * Allows updating any global ripple option at runtime. This makes it possible for developers to disable ripples at runtime. Closes angular#9729 * fixup! feat(ripples): support updating global ripple options at runtime Address feedback
…ular#14705) * feat(ripples): support updating global ripple options at runtime * Allows updating any global ripple option at runtime. This makes it possible for developers to disable ripples at runtime. Closes angular#9729 * fixup! feat(ripples): support updating global ripple options at runtime Address feedback
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Closes #9729